Skip to content

fix-5567 : Update the text in the distribution update mailer ( wording change )#5572

Merged
janeewheatley merged 2 commits into
rubyforgood:mainfrom
mausamp:fix-5567/distribution-mailer-wording-change
Jun 7, 2026
Merged

fix-5567 : Update the text in the distribution update mailer ( wording change )#5572
janeewheatley merged 2 commits into
rubyforgood:mainfrom
mausamp:fix-5567/distribution-mailer-wording-change

Conversation

@mausamp

@mausamp mausamp commented May 22, 2026

Copy link
Copy Markdown
Contributor

Resolves #5567

Description

  • Updated the text in the distribution update mailer as per the text given in the issue.

Type of change

  • Text change

How Has This Been Tested?

  • Updated the distribution update mailer spec to include the updated text.
  • Manually tested the working of the changes in the text.

Screenshots

Before After

|
Screenshot 2026-05-22 at 9 01 29 PM
|
Screenshot 2026-05-22 at 9 02 13 PM
|

@dorner

dorner commented May 29, 2026

Copy link
Copy Markdown
Collaborator

Passes tech review. Over to manual QA. @janeewheatley @ruestitch

@dorner

dorner commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

@mausamp it looks like tests are failing - can you fix?

@mausamp
mausamp requested a review from dorner June 7, 2026 10:05
@janeewheatley

Copy link
Copy Markdown
Collaborator

@dorner @awwaiid This passes QA, but it looks like we need to approval 5 workflows.

I pulled down this code and verified the text change on the distribution email:
image

@janeewheatley
janeewheatley merged commit 3b24d37 into rubyforgood:main Jun 7, 2026
11 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

@mausamp: Your PR fix-5567 : Update the text in the distribution update mailer ( wording change ) is part of today's Human Essentials production release: 2026.06.14.
Thank you very much for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Text on the distribution email when distribution is changed is awkward/grammatically incorrect. Let's change it

3 participants